[15.0][IMP] stock_inventory_lockdown: add hook to decide what locations are excluded from the lockdown#2354
Conversation
AaronHForgeFlow
force-pushed
the
15.0-imp-stock_inventory_lockdown
branch
from
67c9b19 to
cfa6239
Compare
|
Edit: It seems a rebase did the thing 😕 |
AaronHForgeFlow
force-pushed
the
15.0-imp-stock_inventory_lockdown
branch
from
cfa6239 to
86488cb
Compare
| def _location_usage_without_restriction(self, move_line): | ||
| return ( | ||
| move_line.location_dest_id.usage == "inventory" | ||
| or move_line.location_id.usage == "inventory" | ||
| ) |
There was a problem hiding this comment.
wouldn't it be better to pass move_line as self, and then in this method you evaluate self
There was a problem hiding this comment.
Good point! I just did it! 🚀
… excluded from the lockdown some companies may don't want to block all inventoty loss locations and some others will want to block other types of locations
AaronHForgeFlow
force-pushed
the
15.0-imp-stock_inventory_lockdown
branch
from
86488cb to
202bc2a
Compare
|
This PR has the |
LoisRForgeFlow
left a comment
LoisRForgeFlow
left a comment
There was a problem hiding this comment.
👍
/ocabot merge patch
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 4d97fa6. Thanks a lot for contributing to OCA. ❤️ |
5 participants
Some companies may don't want to block all inventory loss locations and some others will want to block other types of locations.
This is just a hook, no change by itself 🙏